Skip to content

refactor(tui): introduce layered state package and boundary docs#166

Merged
minorcell merged 4 commits into
1024XEngineer:mainfrom
creatang:codex/tui-split-01-state-boundary
Apr 7, 2026
Merged

refactor(tui): introduce layered state package and boundary docs#166
minorcell merged 4 commits into
1024XEngineer:mainfrom
creatang:codex/tui-split-01-state-boundary

Conversation

@creatang

@creatang creatang commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator

为 TUI 分层改造建立状态层与分层约束基线。
本次改动:
新增 internal/tui/state/*,拆分 UI/Chat/Runtime 状态与消息常量。
新增 internal/tui/docs/LAYERING.md 和 internal/tui/docs/SKILL.md,明确层级职责与禁止项。
影响范围:仅 internal/tui,不改变入口层行为。
验证:
go test ./internal/tui/... ./internal/app
风险与回滚:
风险低(新增为主);回滚可直接 revert 当前 PR。

@codecov

codecov Bot commented Apr 6, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@phantom5099

Copy link
Copy Markdown
Collaborator

[P2] internal/tui/docs/LAYERING.md:12-16
文档把 L1 入口层写成了 cmd/tui/,但仓库实际入口是 cmd/neocode。这会直接误导后续分层迁移的落点。

[P2] internal/tui/docs/LAYERING.md:70-94
文档把 bootstrap/core/components/services/infra 目录骨架、bootstrap.Build、internal/tui/core/runtime_bridge.go,以及 GetRunSnapshot / GetSessionContext / GetSessionUsage / GetRunUsa

@creatang

creatang commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator Author

[P2] internal/tui/docs/LAYERING.md:12-16 文档把 L1 入口层写成了 cmd/tui/,但仓库实际入口是 cmd/neocode。这会直接误导后续分层迁移的落点。

[P2] internal/tui/docs/LAYERING.md:70-94 文档把 bootstrap/core/components/services/infra 目录骨架、bootstrap.Build、internal/tui/core/runtime_bridge.go,以及 GetRunSnapshot / GetSessionContext / GetSessionUsage / GetRunUsa

[P2] internal/tui/docs/LAYERING.md:12-16 文档把 L1 入口层写成了 cmd/tui/,但仓库实际入口是 cmd/neocode。这会直接误导后续分层迁移的落点。

[P2] internal/tui/docs/LAYERING.md:70-94 文档把 bootstrap/core/components/services/infra 目录骨架、bootstrap.Build、internal/tui/core/runtime_bridge.go,以及 GetRunSnapshot / GetSessionContext / GetSessionUsage / GetRunUsa

这个入口是指tui入口,我调整文档吧,第二个是什么意思呢?没有看懂

@wynxing

wynxing commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator

这个SKILL.md是干嘛的?这个不是给ai调用的吗

@creatang creatang force-pushed the codex/tui-split-01-state-boundary branch from 871015c to 56384b2 Compare April 6, 2026 13:18
@creatang creatang force-pushed the codex/tui-split-01-state-boundary branch from 56384b2 to 9b52492 Compare April 6, 2026 15:19
@minorcell minorcell merged commit 33ee592 into 1024XEngineer:main Apr 7, 2026
2 checks passed
@creatang creatang deleted the codex/tui-split-01-state-boundary branch April 10, 2026 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants